Skip to content

I want more time.... It will probably be better to learn more before sinking more effort into this, though. - #9

Open
adellhk wants to merge 14 commits into
sf-squirrels-2015:masterfrom
adellhk:master
Open

I want more time.... It will probably be better to learn more before sinking more effort into this, though.#9
adellhk wants to merge 14 commits into
sf-squirrels-2015:masterfrom
adellhk:master

Conversation

@adellhk

@adellhk adellhk commented Feb 9, 2015

Copy link
Copy Markdown

No description provided.

Comment thread app/helpers/session.rb

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is a global function which is a bad practice. All helpers should be wrapped in

helpers do
  
end

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants